Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Container update #594

Closed
wants to merge 68 commits into from
Closed

Container update #594

wants to merge 68 commits into from

Conversation

sashakames
Copy link
Collaborator

@sashakames sashakames commented Jan 17, 2024

Description

This PR will be a general update for containers.

  • Add runtime configuration for frontend
  • Automate container build/release to ghcr.io

Todo before merge:

  • Fix container workflow trigger
  • Revert 36b779c

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Local Pre-commit Checks
  • CI/CD Build

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • If applicable - I have commented my code, particularly in hard-to-understand areas
  • If applicable - I have made corresponding changes to the documentation
  • If applicable - I have added tests that prove my fix is effective or that my feature works
  • If applicable - New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

downiec and others added 30 commits September 22, 2022 18:22
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* enable settings on the backend for urls

* add the settings for workflow

* add missing import

* add newline
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…environment variables. Otherwise local build of backend will fail.
…ests. Removed unneccessary type checking in the import headers of the test models. Updated coverage for some files that didn't have 100% coverage. Updated some django dependencies after testing to make sure nothing broke. Updated some deprecated imports to remove warnings. Updated pytest and coverage libraries.
…h the application from users perspective was working fine)
…nges that came with react-router-dom v6. Tests are passing now.
* Refactored the joyrideTutorials in order to reduce the chances for errors with misspelling target names. Created unique target calssnames to simplify the process of adding and using targets. Removed the chance of duplicate target names causing errors etc. Added a test startup component that should show when first opening metagrid. Startup window still needs some work to improve visuals.

* Added a welcome and changelog template component to use with thh startup display. Made the startup display change based on existing version so that a new changelog will appear when version is changed. Modified the startup component behavior to hadle a welcome and changelog template based on whether its first-time startup or not.

* Updated django-all-auth to 0.54.0

* Upgraded the antd library to latest version. Updated components to work with the latests antd library and added some styling to correct some undesired changes. Added a temporary drawer component which can be opened by clicking a new message button in the top-right navbar. Fixed some styling issues with window resizing to make the top navbar work better with smaller windows and improve visual behavior. Removed redundant components that are no longer needed with the new antd library (which is now working with typescript well). Created a popup window that shows at the start for new users. Created a startup template system that allows different templates to be used for displaying popup messages. More fixed needed for tests and the welcome popup tutorials need work.

* Worked to correct issues with tests hanging by reverting back the antd library. Also updated to node version 18. Restored previously deleted files and will remove them later when all tests are passing. Still need to correct tests to pass. Still need to complete messaging feature and update tutorial.

* Updated tests so that they pass, coverage will still be needed.

* Updated the Startup template functionality and added actions that can be triggered by the templates. Updated the welcome message buttons to work properly and select between pages to start tutorials.

* Removed redundant and unneccessary components as they are now directly taken from antd library. Removed commented out import statements.

* Refactored some names and added messages for the right drawer. Added ability to create messages for the message bar and provide content from markdown files. Updated the changelog to have useful information on latest changes. Need to troubleshoot tests and complete coverage.

* Added markdown file support for the messages. Did refactoring so messages on the right drawer and popups will show the markdown. Created new markdown files with version info and example message. Added flexibility to popup so that styles can be modified on a per message basis. Modified the changelog template and types.

* Created a new welcome tour which will show users the help buttons to encourage them to use them if they have issues. Updated the navbar tour to include the new news button. Created new test files and updated tests to bring back 100% text coverage. Removed obsolete tour target related files, as they've been updated with target object class. Troubleshooted and ran tests to fix some bugs and added more functionality for jest tests. Deleted test markdown files and updated message markdown slightly.

* modify the messages for release (#522)

---------

Co-authored-by: Sasha Ames <sashakames@users.noreply.github.com>
* Refactored the joyrideTutorials in order to reduce the chances for errors with misspelling target names. Created unique target calssnames to simplify the process of adding and using targets. Removed the chance of duplicate target names causing errors etc. Added a test startup component that should show when first opening metagrid. Startup window still needs some work to improve visuals.

* Added a welcome and changelog template component to use with thh startup display. Made the startup display change based on existing version so that a new changelog will appear when version is changed. Modified the startup component behavior to hadle a welcome and changelog template based on whether its first-time startup or not.

* Updated django-all-auth to 0.54.0

* Upgraded the antd library to latest version. Updated components to work with the latests antd library and added some styling to correct some undesired changes. Added a temporary drawer component which can be opened by clicking a new message button in the top-right navbar. Fixed some styling issues with window resizing to make the top navbar work better with smaller windows and improve visual behavior. Removed redundant components that are no longer needed with the new antd library (which is now working with typescript well). Created a popup window that shows at the start for new users. Created a startup template system that allows different templates to be used for displaying popup messages. More fixed needed for tests and the welcome popup tutorials need work.

* Worked to correct issues with tests hanging by reverting back the antd library. Also updated to node version 18. Restored previously deleted files and will remove them later when all tests are passing. Still need to correct tests to pass. Still need to complete messaging feature and update tutorial.

* Updated tests so that they pass, coverage will still be needed.

* Updated the Startup template functionality and added actions that can be triggered by the templates. Updated the welcome message buttons to work properly and select between pages to start tutorials.

* Removed redundant and unneccessary components as they are now directly taken from antd library. Removed commented out import statements.

* Refactored some names and added messages for the right drawer. Added ability to create messages for the message bar and provide content from markdown files. Updated the changelog to have useful information on latest changes. Need to troubleshoot tests and complete coverage.

* Added markdown file support for the messages. Did refactoring so messages on the right drawer and popups will show the markdown. Created new markdown files with version info and example message. Added flexibility to popup so that styles can be modified on a per message basis. Modified the changelog template and types.

* Created a new welcome tour which will show users the help buttons to encourage them to use them if they have issues. Updated the navbar tour to include the new news button. Created new test files and updated tests to bring back 100% text coverage. Removed obsolete tour target related files, as they've been updated with target object class. Troubleshooted and ran tests to fix some bugs and added more functionality for jest tests. Deleted test markdown files and updated message markdown slightly.

* modify the messages for release (#522)

* Updated the node status information alert so that text displays if there is an api error.

* Updated Django to 4.1.9

* Fixed a test regarding the changes to the error message.

---------

Co-authored-by: Sasha Ames <sashakames@users.noreply.github.com>
…null items don't cause an error and prevent the site from displaying.
downiec and others added 26 commits June 22, 2023 20:26
…ity as Globus Transfers instead of Globus Downloads. Added more content to the changelog for this update.
…ransfer tasks that were recently submitted. Created the logic and styling needed to make the tasks populate after each successful submission. I fixed some styling issues with the table sizing (when the window width is smaller. I fixed issue where the state of the useDefaultEndpoint radio options wasn't showing correctly. Extended the Button component to accept style props. Did some refactoring to move the Globus related types to the Globus component directory. Created new recoil stat for task ID's to make tthe summary page and download components share and update the tasks list.
…xed it by making sure the message displays first using the async await call.
… globus ready filters. Didn't add tutorial steps for the submit task history, since that would require transfers to exist. May update that in future.
…use errors), modified the transfer messaging to wait for the message to finish before allwing more transfers. Attempted to add disable logic for transfer button to improve reliability, however may be changed later if needed. Updated the messages content and added more information to changelog message. Added tooltips for the endpoint settings on Globus transfer page (in place of tutorial steps).
…s, resolved issue with blank error messages. If error.message is empty, there will be a generic message displayed: unknown error occurred. Changed the breadcrumbs organization so that search page is clearly identified as home page, and updated home button to redirect to the home page to resolve issue 533
* info on Globus Transfer

* Update index.tsx

Added styling fixes and updated.

---------

Co-authored-by: Carlos Downie <42552189+downiec@users.noreply.github.com>
…ocation of notifications to be below the top navigation bar. Functions allow messages to be standardized and empty error messages can give the same generic error message when needed. Styling can be standardized in one place.
… aims2 and updated the references to the docker-compose command to be 'docker compose' (with a space) to match the newer docker compose recommendations. Resolved an issue where the items in the cart were not correctly displaying the node status and were instead showing a question mark. Updated traefik version to 2.10 and updated the config file.
* trying to add the link

* Adjusted location of the federated nodes link and added the link to the ESGF logo as well.

* Updated django to latest version, updated the test backend.tml to include the new SOLR_URL setting.

* Updated postgres library

* Updated psycopg-binary to see if tests will pass

* update link hostname

* Updated psycopg to include pyscopg 3

* Upgraded various backend packages and downgraded psycopg3 to psycopg2

* Updated the postgres version used in tessts, to match the current postress version being used in production and local.

---------

Co-authored-by: Carlos Downie <42552189+downiec@users.noreply.github.com>
… api_proxy temp storage functions. Created test files for the globus download backend functions. Tests for the download and server related code were left untested due to difficulty in making a unit test for a globus transfer (ignored coverage instead). Created tests to bring backend coverage back to 100%
@jasonb5 jasonb5 mentioned this pull request Feb 16, 2024
13 tasks
@jasonb5
Copy link
Collaborator

jasonb5 commented Feb 16, 2024

Closing, see new PR #608

@jasonb5 jasonb5 closed this Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants